Skip to content

Fixed in Intellij 13#13

Open
peterox wants to merge 2 commits into
crc83:masterfrom
peterox:master
Open

Fixed in Intellij 13#13
peterox wants to merge 2 commits into
crc83:masterfrom
peterox:master

Conversation

@peterox

@peterox peterox commented Mar 28, 2014

Copy link
Copy Markdown

Added better connection pooling to RallyRestApi.

@crc83

crc83 commented Jun 28, 2014

Copy link
Copy Markdown
Owner

I'm really doubt if we need this at all. This connection is not used so often and amount of data is not so big.

@peterox

peterox commented Jul 4, 2014

Copy link
Copy Markdown
Author

This was needed so that a different connection is used by the server config dialog and any background tasks that use the repository. Without this fix the single connection created is cleaned up and the background tasks fail.

@crc83

crc83 commented Oct 20, 2014

Copy link
Copy Markdown
Owner

Too much file changed. Could you please pull only those related to an issue?

@cweagans

cweagans commented Nov 5, 2015

Copy link
Copy Markdown

FWIW, I started rerolling this for IntelliJ 15 today, and it causes a ton of files to change. Probably pretty unavoidable.

The connection pooling stuff could be it's own PR, though.

@crc83

crc83 commented Jan 9, 2019

Copy link
Copy Markdown
Owner

Sorry for ages of silence. I'll integrate these changes, but as a commit from my repo).
Peter, I'll mention you in class header.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants